Skip to content

Improve CI pipeline and image maintenance - #619

Open
kikoveiga wants to merge 2 commits into
developfrom
kikoveiga/remove-woke-ci-cleanup
Open

Improve CI pipeline and image maintenance#619
kikoveiga wants to merge 2 commits into
developfrom
kikoveiga/remove-woke-ci-cleanup

Conversation

@kikoveiga

@kikoveiga kikoveiga commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replaces the shared static-analysis pipeline with repository-owned Detekt, Ktlint, Android Lint, license, and generated-file checks aligned with dd-sdk-android.
  • Centralizes Ktlint configuration, standardizes source headers, and converts the remaining Groovy build script to Kotlin DSL.
  • Replaces JaCoCo and its flaky per-module coverage threshold with Kover 0.9.8 while preserving module-level Datadog coverage uploads.
  • Streamlines the CI image and build context, embeds the checksum-verified Java tracer, updates Android tooling, and supports architecture-aware Datadog CI installation.
  • Aligns pipeline stage naming and auto-cancels superseded short-lived branch pipelines while protecting stateful jobs.

Validation

  • ./gradlew koverXmlReport :build-config:test :dd-sdk-android-gradle-plugin:checkThirdPartyLicences
  • ./gradlew ktlintCheck :build-config:ktlintCheck detekt --continue
  • ./gradlew checkGeneratedFiles --no-configuration-cache
  • ./gradlew lintRelease --continue
  • ./gradlew :samples:variants:tasks
  • GitLab YAML and diff formatting checks

Notes

  • These are CI and build-tooling changes; no plugin version bump is required.
  • The manual ci-image job must publish and sign image 19.
  • The full image build was not run locally because Docker was unavailable.

@kikoveiga
kikoveiga force-pushed the kikoveiga/remove-woke-ci-cleanup branch 2 times, most recently from b67a8bb to 59f101e Compare September 1, 2026 13:36
@kikoveiga kikoveiga changed the title Remove Woke and streamline the CI image Improve CI pipeline and image maintenance Sep 1, 2026
@kikoveiga
kikoveiga force-pushed the kikoveiga/remove-woke-ci-cleanup branch 4 times, most recently from 33a683a to c8bddf5 Compare September 1, 2026 14:36
@datadog-prod-us1-5

datadog-prod-us1-5 Bot commented Sep 2, 2026

Copy link
Copy Markdown

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 62.49% (-14.13%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 92e35f0 | Docs | View more details | Give us feedback!

@kikoveiga
kikoveiga force-pushed the kikoveiga/remove-woke-ci-cleanup branch 4 times, most recently from 79c9948 to f4639eb Compare September 2, 2026 13:34
@kikoveiga
kikoveiga force-pushed the kikoveiga/remove-woke-ci-cleanup branch from f4639eb to 2cfab7e Compare September 2, 2026 13:35
@kikoveiga
kikoveiga marked this pull request as ready for review September 2, 2026 13:35
@kikoveiga
kikoveiga requested review from a team as code owners September 2, 2026 13:35
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-03T09:06:45.267464Z 92e35f0 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Comment thread ci/Dockerfile.gitlab Outdated
Comment thread ci/Dockerfile.gitlab Outdated
Comment thread ci/pipelines/default-pipeline.yml
Comment thread ci/pipelines/default-pipeline.yml Outdated
Comment thread ci/pipelines/default-pipeline.yml
Comment thread samples/variants/build.gradle

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 23ef5bb776

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ci/pipelines/default-pipeline.yml
Comment thread ci/pipelines/default-pipeline.yml Outdated
@kikoveiga
kikoveiga force-pushed the kikoveiga/remove-woke-ci-cleanup branch from 23ef5bb to cf301c0 Compare September 2, 2026 15:39
@kikoveiga
kikoveiga requested a review from 0xnm September 2, 2026 15:42
@kikoveiga
kikoveiga force-pushed the kikoveiga/remove-woke-ci-cleanup branch from cf301c0 to 91ddad5 Compare September 2, 2026 16:03
0xnm
0xnm previously approved these changes Sep 3, 2026
@kikoveiga
kikoveiga force-pushed the kikoveiga/remove-woke-ci-cleanup branch from 91ddad5 to 92e35f0 Compare September 3, 2026 09:00
@kikoveiga
kikoveiga requested a review from 0xnm September 3, 2026 09:00

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 92e35f0fe0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants